Skip to content

Conversation

@cgturner1
Copy link
Contributor

Separates platform entries in the configuration array to reduce reduce the number of platfroms run per runner as we are approaching size limits from building too many at a time.

Signed-off-by: Christopher Turner [email protected]

@lgirdwood
Copy link
Member

Hopefully see more green in CI results with this.

@cgturner1 cgturner1 force-pushed the fix-zephyr-bld branch 3 times, most recently from f3aede3 to b9be260 Compare January 14, 2026 17:41
Separates platform entries in the configuration array to
reduce reduce the number of platfroms run per runner as
we are approaching size limits from building too many at a time.

Signed-off-by: Christopher Turner <[email protected]>
@cgturner1 cgturner1 marked this pull request as ready for review January 14, 2026 18:11
Copilot AI review requested due to automatic review settings January 14, 2026 18:11
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR splits platform matrix entries in the Zephyr CI workflow to reduce the number of platforms built per runner, addressing size limits from building too many platforms simultaneously.

Changes:

  • Split grouped platform entries in IPC_platforms matrix into separate lines
  • Split grouped platform entries in platforms matrix into separate lines
  • Updated test validation logic to match the new platform split

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

# Temporary testbed for Zephyr development.
tgl tgl-h,
tgl,
tgl-h
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing trailing comma after 'tgl-h'. This inconsistency with other array entries could cause parsing issues.

Suggested change
tgl-h
tgl-h,

Copilot uses AI. Check for mistakes.
# legacy
tgl tgl-h,
tgl,
tgl-h
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing trailing comma after 'tgl-h'. This inconsistency with other array entries could cause parsing issues.

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

@kv2019i kv2019i left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cgturner1 !

@kv2019i
Copy link
Collaborator

kv2019i commented Jan 15, 2026

The Jenkins CI build fail https://sof-ci.01.org/sofpr/PR10482/build18513/build/index.html is not realted to this PR, ignoring that and merging.

@kv2019i kv2019i merged commit b1a7c44 into thesofproject:main Jan 15, 2026
45 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants